Coverity can't trust GetOpacity() to return a constant value, as it's used three times in this function, it's better to cache it and make coverity happy than call the same function three times (unless it's actually dynamic, in which case coverity says you can use uninitialized data). 128 ContainerLayerOGL::RenderLayer(int aPreviousFrameBuffer) 133 GLuint containerSurface; 140 if (GetOpacity() != 1.0) { 182 } else { 183 frameBuffer = aPreviousFrameBuffer; 184 } 205 if (GetOpacity() != 1.0) { 235 gl()->fBindTexture(LOCAL_GL_TEXTURE_2D, containerSurface); 237 rgbProgram->SetLayerOpacity(GetOpacity());